Skip to content

refs #13789 - --project-configuration had no effect with wrapped Visual Studio project#7476

Merged
chrchr-github merged 2 commits intocppcheck-opensource:mainfrom
firewave:vs-proj-cfg
Apr 23, 2025
Merged

refs #13789 - --project-configuration had no effect with wrapped Visual Studio project#7476
chrchr-github merged 2 commits intocppcheck-opensource:mainfrom
firewave:vs-proj-cfg

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

Comment on lines +209 to +215
@pytest.mark.xfail # TODO: no source files found
def test_cppcheck_project_local_path_select_one():
__test_cppcheck_project_local_path(['--project-configuration=Release|Win32'], 'Release|Win32')

@pytest.mark.xfail # TODO: no source files found
def test_cppcheck_project_local_path_select_one_multiple():
__test_cppcheck_project_local_path(['--project-configuration=Debug|Win32', '--project-configuration=Release|Win32'], 'Release|Win32')
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have not looked into why it does not find source files. I will file a separate ticket about that later.

Part of this PR is needed for some other changes I am currently working on.

@firewave firewave marked this pull request as draft April 19, 2025 22:12
@firewave firewave marked this pull request as ready for review April 19, 2025 22:12
@firewave firewave changed the title fixed #13789 - --project-configuration had no effect with wrapped Visual Studio project refs #13789 - --project-configuration had no effect with wrapped Visual Studio project Apr 19, 2025
@chrchr-github chrchr-github merged commit 23b0c9a into cppcheck-opensource:main Apr 23, 2025
53 checks passed
@firewave firewave deleted the vs-proj-cfg branch April 23, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants